-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Drop Node 6 support. #45
Conversation
Fix linting to avoid CI breaking
@@ -54,8 +55,7 @@ | |||
"decimal.js": "^10.2.0", | |||
"js-yaml": "^3.13.1", | |||
"json-schema-deref-sync": "^0.13.0", | |||
"object.values": "^1.1.1", | |||
"swagger-parser": "^6.0.5" | |||
"swagger-parser": "^9.0.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is an important benefit of bumping Node version, up-to-date swagger-parser no longer brings in core-js as a transitive dependency, which should help solve PayU/openapi-validator-middleware#116
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kibertoad , if we're going to release a breaking change, i'd like to also handle this together #39
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, I'll work on that, but probably they can be merged separately.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kobik Do you think that this change makes sense? |
yes, i think you have a point here. too bad they still haven 't implemented sync file loading. |
Also fixes linting to avoid CI breaking.
Considering that even Node 8 is already deprecated, it might make sense to drop support for Node 6, remove shims, upgrade to swagger-parser version that also got rid of shims, and bump major version.
Related to PayU/openapi-validator-middleware#116